Merge branch 'flatpak-tweaks' into 'master'

Flatpak tweaks

See merge request GNOME/gtk!3187

Merge branch 'ci-fedora-33' into 'master'

ci: Update the Fedora image to F33

See merge request GNOME/gtk!3188

Merge branch 'wip/baedert/for-master' into 'master'

Wip/baedert/for master

See merge request GNOME/gtk!3189

nativedialog: Remove unused run state

build: Check for mallinfo2 instead of mallinfo

Similar to https://github.com/llvm/llvm-project/commit/e89fcbfad6a3edb203ebdc2b1ead275fe909281a
but the tests we use mallinfo() in really aren't important to not worth
the effort to differentiate between mallinfo and mallinfo2. Only check
for mallinfo2().

flatpak: Try harder to disable sassc

Add devel styling to nightly flatpaks

Pass -Dprofile=devel to the gtk build for our nightly
flatpaks, so they pick up devel styling.

demos: Add devel styling

Add a -Dprofile=devel meson option, and add some visual
hints to the demos that you are running a nightly build.

flatpak: No need to build sassc anymore

We should be able to do a gtk build without rebuilding
css nowadays.

ci: Update the Fedora image to F33

Merge branch 'fix-network-places' into 'master'

placesview: Avoid use-after-free

See merge request GNOME/gtk!3186

Merge branch 'wip/jimmac/tiled-shadow-fix-fix-gtk4' into 'master'

Adwaita: bring borders for tiled

Closes #3661

See merge request GNOME/gtk!3184

Merge branch 'ebassi/memdup2' into 'master'

Use g_memdup2()

See merge request GNOME/gtk!3157

tests: Move from g_memdup() to g_memdup2()

The g_memdup() function is replaced by a safer version in newer versions
of GLib.

Ensure we don't overflow when using g_memdup2()

When we turn integers into size_t we should check we're not going to
make a mess.

win32: Move from g_memdup() to g_memdup2()

Use `size_t` and `gsize` consistently for the length argument in every
function taking a bytes array and a length.

gtk: Move from g_memdup() to g_memdup2()

The g_memdup() function is replaced by a safer version in newer versions
of GLib.

gsk: Move from g_memdup() to g_memdup2()

The g_memdup() function is replaced by a safer version in newer versions
of GLib.

gdk: Move from g_memdup() to g_memdup2()

The g_memdup() function is replaced by a safer version in newer versions
of GLib.

x11: Move from g_memdup() to g_memdup2()

The g_memdup() function is replaced by a safer version in newer versions
of GLib.

wayland: Move from g_memdup() to g_memdup2()

The g_memdup() function is replaced by a safer version in newer versions
of GLib.

macos: Move from g_memdup() to g_memdup2()

The g_memdup() function is replaced by a safer version in newer versions
of GLib.

broadway: Move from g_memdup() to g_memdup2()

The g_memdup() function is replaced by a safer version in newer versions
of GLib.

Adwaita: bring borders for tiled

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/3661

placesview: Avoid use-after-free

The places view has questionable code for handling
'destroy', which seems to trigger use-after-free in
some cases. Clean this up a bit by acknowledging the
cancellation fully, rather than relying on the
destroyed flag.

Add compatibility shim for g_memdup2()

We don't necessarily want to depend on a newer version of GLib, given
that we're not really using g_memdup() in the wrong way.

Merge branch 'wip/chergert/inline-transform-category' into 'master'

gsk: move transform structure to private.h

See merge request GNOME/gtk!3182

Merge branch 'disable-macos-ci' into 'master'

ci: Temporarily disable macOS CI job as runner is offline

See merge request GNOME/gtk!3183

Merge branch 'fix-filter-value' into 'master'

css: Avoid an invalid read

See merge request GNOME/gtk!3181

ci: Temporarily disable macOS CI job as runner is offline

See https://gitlab.gnome.org/Infrastructure/GitLab/-/issues/495

Merge branch 'wip/chergert/fix-unused' into 'master'

snapshot: fix warning when asserts are disabled

See merge request GNOME/gtk!3180

gsk: move transform structure to private.h

This allows using the category without having to call a function from the
hot paths of the renderer.

snapshot: fix warning when asserts are disabled

css: Avoid an invalid read

This was broken in ea7185bdb1ba423495340a9b880e619a861064fd.

Pointed out by Christian Hergert.

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!3177

iconbrowser: Use the right icon

We have an app icon, lets use it.

print-editor: Use the non-devel icon

We are no longer a nightly print editor.

node-editor: Use the non-devel icon

We're now a respectable, stable node editor.

Merge branch 'fix-clipped-texture' into 'master'

gl: Fixed clipped offscreen rendering

Closes #3651

See merge request GNOME/gtk!3175

gl: Fixed clipped offscreen rendering

When we are rendering a texture node to an offscreen,
and we have a clip, we must force the offscreen rendering.
Otherwise, the code will notice: Hey, it already is a texture
node, so no need to render it to a texture again. But when
clipping is involved, that is exactly what we want to do.

Testcase included.

Fixes: #3651

Merge branch 'wip/chergert/fix-maximized-shadows' into 'master'

adwaita: disable shadows when window is maximized or fullscreen

See merge request GNOME/gtk!3173

Adwaita: don't cast shadows for halt-tiled windows

- while the case of a single half tiled window casting shadow is valid,
  the main use case for half tiled windows is when they have a neighbor.
  The :backdrop should be enough of a focus indicator and we avoid shadow
  casting on neighbors that are on the same Z level.

adwaita: disable shadows when window is maximized or fullscreen

These shadows cause a significant draw performance drop for maximized
windows. Disabling them increases the chances we can have faster scroll
performance of text.

There is some risk here for systems where they have a dock and you expect
the shadow to draw beneath that dock for transparency reasons.

Merge branch 'win32-drop-formats-leak' into 'master'

win32: Fix some leaks

See merge request GNOME/gtk!3167

Merge branch 'wip/chergert/fix-bgr8-uploads' into 'master'

gl: handle B8G8R8 texture data when uploading

See merge request GNOME/gtk!3171

gl: handle B8G8R8 texture data when uploading

We do not need to translate this on the CPU when we can instead push it
to the GPU in the same format and allow it to swizzle.

This fixes a huge number of memory allocations found while uploading the
GTK animation in widget-factory.

Merge branch 'wip/chergert/macos-perf-fixes' into 'master'

macos: various OpenGL performance fixes

See merge request GNOME/gtk!3170

macos: improve use of swap rectangles with OpenGL

macos: only flush when not attached

This makes it so we only flush the context for the NSView, not the context
that is the center of our center of the GL context spokes.

macos: mark region as unlikely

gsk: make gsk_render_node_get_node_type() use const

There isn't any state to modify in the type so we can use const here.
Doing so allows some of the renderer code to use const across a
number of functions so that repeated calls are elided if inlined.

gsk: make render node getters pure

These do not do modify anything so they can be marked as pure to
potentially ellide calls. Since they do dereference, I do not believe
we can make them const although that is unclear since we could technically
just return a pointer + offset. Therefore it *might* be possible to also
make these G_GNUC_CONST.

gsk: make render node getters const

This also removes the return if fail macros from these as a good portion
of them didn't have them anyway. I think it's fair to say that access to
these incorrectly is a programmer error.

It significantly reduces the amount of code generated into generally a
movss,ret.

profiler: allow using gdk profiler functions without braces

Just compiling these out means you have to write code slightly differently
so that you don't end up with "if ();" afterwards.

This adds a "do {} while (0)" so that we're still semantically a statement
but will also compile out.

Merge branch 'wip/exalm/headerbar' into 'master'

headerbar: Don't use gtk_widget_unparent() with GtkBox

See merge request GNOME/gtk!3169

headerbar: Null-check boxes when removing window controls

Prevent a crash when show-title-buttons value changes to FALSE during
destruction.

headerbar: Don't use gtk_widget_unparent() with GtkBox

gtk_widget_unparent() is for widget implementations, it's not supposed to
be used from outside.

Merge branch 'printing-temporary-queues-gtk4' into 'master'

printing: Create temporary queues for Avahi printers

See merge request GNOME/gtk!3160

Merge branch 'focus' into 'master'

listbox: Fix handling of non-focusable rows

Closes #3633

See merge request GNOME/gtk!3152

Merge branch 'fix-x11-drop' into 'master'

x11: More dnd fixes

Closes #3654

See merge request GNOME/gtk!3166

win32: Fix some leaks

My reading of the code is that gdk_drop_new() is not
consuming the content formats it is given, so the caller
must not pass a new reference.

Needs testing on Windows.

x11: More dnd fixes

Whenever we communicate targets, we need to the union, otherwise
we don't tell the other side about our serialization. This makes
drops of images from gtk4-icon-browser to gimp and libreoffice
succeed in transferring data.

Fixes: #3654

Merge branch 'fix-x11-drop' into 'master'

Fix x11 drop

Closes #3652

See merge request GNOME/gtk!3165

x11: Pass mimetypes when creating output stream

When creating the output stream for a drop, we must
pass the mimetypes we support, otherwise the picking
of the right handler does not work.

Fixes: #3652

x11: Fix a memory leak

We ref the content formats here, so we better unref it
when done.

x11: Some code cleanups

Fix some glaring misindentation.

Merge branch 'matthiasc/for-master' into 'master'

Implement drop-shadow css filter

See merge request GNOME/gtk!3163

Add tests for the css filter property

Test that we can parse filters and that we
don't accept invalid filters.

cssfiltervalue: Reject invalid filters

The Filter Effects Spec doesn't allow negative values
for most of the filter parameters.

cssfiltervalue: Fix blur filter interpretation

According to https://www.w3.org/TR/filter-effects-1/,
the length passed to blur() is the standard deviation,
and according to https://www.w3.org/TR/css-backgrounds-3/#shadow-blur
the blur radius is twice the standard deviation.

cssfiltervalue: Implement drop-shadows

We have all the pieces, so this is surprisingly easy.

cssshadowvalue: Add a 'filter mode'

Shadow values created by gtk_css_shadow_value_new_filter or
gtk_css_shadow_value_parse_filter interpret their radius value
as standard deviation. Add a flag for this mode, and use it
where necessary.

cssshadowvalue: Add a parsing function

Add a variant of the parse function that parses
just a single (non-box) shadow, as required for
the drop-shadow filter.

cssshadowvalue: Add gtk_css_shadow_value_pop_snapshot

This is the counterpart ot gtk_css_shadow_value_push_snapshot.

To make this easy, move the determination whether we need a
shadow out of the push function and save it.

cssshadowvalue: Drop the underscore

Rename _gtk_css_shadow_value_parse to drop the underscore.
It was the only underscore-prefixed function in this file.

Updated Czech translation

Add tests for the css filter property

Test that we can parse filters.

css: Implement the drop-shadow filter

We have all the pieces, so this is surprisingly easy.

cssshadowvalue: Add a parsing function

Add a variant of the parse function that parses
just a single (non-box) shadow, as required for
the drop-shadow filter.

Merge branch 'module-ext' into 'master'

meson: use correct module file extension on macOS

Closes #3645

See merge request GNOME/gtk!3162

meson: use correct module file extension on macOS

GModule requires the .so file extension on macOS for historic reasons.
However Meson defaults to .dylib for modules, so we need to override
it to get the correct extension.

Fixes #3645.

printing: Show all Avahi advertised printers

This commit unsubscribes CUPS backend from a DBus
signal in idle when listening for new items on Avahi.

Since GDBus emits gathered signals in idle while
checking whether the signal has been unsubscribed
it could happen that a signal was not processed
because it was removed from hash table of
subscribed signals.
This caused the situation where printers advertised
on Avahi were not listed in CUPS backend sometimes.

We need those signals since this happens when switching
from a general subscription which listens to signals
for all Avahi services to a specific one which listens
to just _ipp._tcp and _ipps._tcp (chicken and egg problem).

printing: Create temporary queues for Avahi printers

This change extends set of Avahi advertised printers which
works with Gtk's CUPS print backend.

It creates a temporary queue (local printer) for each
Avahi printer in CUPS instead of accessing them directly
(via CUPS library).

This makes some printers work which did not work before and
also gives users more options to change in the print dialog.

This also changes naming of printers to be in accordance with CUPS.
It uses '_' instead of '-' and has hostname appended for CUPS remote
printers.

Merge branch 'matthiasc/for-master' into 'master'

iconbrowser: Make image dnd work again

Closes #3648

See merge request GNOME/gtk!3159

Merge branch 'master' into 'master'

Improve the docs of GtkWidget and GtkGrid

See merge request GNOME/gtk!2946

iconbrowser: Make image dnd work again

We need to drag a texture, not a paintable.

Fixes: #3648

Merge branch 'matthiasc/for-master' into 'master'

docs: Rewrite the long description for GtkDialog

Closes #3646

See merge request GNOME/gtk!3156

Merge branch 'doc-typo' into 'master'

gtkshow: Fix doc typo

See merge request GNOME/gtk!3158

gtkshow: Fix doc typo

docs: Rewrite the long description for GtkDialog

As was pointed out in #3646, some of the content here
was a bit outdated.

Fixes: #3646

Merge branch 'matthiasc/for-master' into 'master'

More work on css transition tests

See merge request GNOME/gtk!3154

Drop the installed test for now

It fails in ci, and I have no idea why.

Merge branch 'pvs-fixes' into 'master'

Pvs fixes

See merge request GNOME/gtk!3155

treemodelfilter: Drop unreachable code

We never get here. The compiler says so.

Pointed out in https://www.viva64.com/en/b/0793/

css: Drop a bit of unreachable code

We never get here. The compiler says so.

Pointed out in https://www.viva64.com/en/b/0793/

a11y: Avoid out-of-bounds access

Don't use the index before we've checked its good.

Pointed out in https://www.viva64.com/en/b/0793/

gdk: Remove a redundant check

We already know desktop_notification_id is not NULL.

Pointed out in https://www.viva64.com/en/b/0793/

messagedialog: Don't initialize twice

We don't need to set these fields more than once.

Pointed out in https://www.viva64.com/en/b/0793/

composetable: Remove a redundant check

We already know seq_index is not NULL here.

Pointed out in https://www.viva64.com/en/b/0793/

x11: A case of argument order confusion

translate_keysym was expecting its arguments the
other way around.

Pointed out in https://www.viva64.com/en/b/0793/